Skip to content

Downport the netlink patch to libteam1.26. Increase netlink buffers#2822

Merged
pavel-shirshov merged 1 commit intomasterfrom
pavelsh/libteam_buf
Apr 25, 2019
Merged

Downport the netlink patch to libteam1.26. Increase netlink buffers#2822
pavel-shirshov merged 1 commit intomasterfrom
pavelsh/libteam_buf

Conversation

@pavel-shirshov
Copy link
Contributor

@pavel-shirshov pavel-shirshov commented Apr 25, 2019

- What I did

  1. Downport the netlink patch:
    jpirko/libteam@046fb6b#diff-2534a34d63e1982340c4691ab57c193f
  2. Increase default buffer size from 98304 to 983040

I did that because I found: PortChannel115: Failed to send options change via netlink (err -105) In kernel.log. I checked kernel sources and found that error 105 means: ENOBUFS.
From netlink man page

   However, reliable transmissions from kernel to user are impossible in
   any case.  The kernel can't send a  netlink  message  if  the  socket
   buffer  is  full:  the message will be dropped and the kernel and the
   user-space process will no longer have the same view of kernel state.
   It  is  up  to  the  application to detect when this happens (via the
   ENOBUFS error returned by recvmsg(2)) and resynchronize.

So I applied a patch witch increases size of the netlink buffers.

- How I did it

- How to verify it
Build an image and run on DUT with 32 LACP interfaces

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@pavel-shirshov pavel-shirshov merged commit 452e75c into master Apr 25, 2019
@pavel-shirshov pavel-shirshov deleted the pavelsh/libteam_buf branch April 25, 2019 23:16
MichelMoriniaux pushed a commit to criteo-forks/sonic-buildimage that referenced this pull request May 28, 2019
mssonicbld added a commit that referenced this pull request Jun 16, 2023
…lly (#15468)

#### Why I did it
src/sonic-swss
```
* 87e0b08 - (HEAD -> master, origin/master, origin/HEAD) [portsorch]: Enhancing SWSS OA logs to capture host_tx_ready change events (#2822) (11 hours ago) [mihirpat1]
* c7e52a0 - [subinterface]: Fix admin state handling. (#2806) (34 hours ago) [Nazarii Hnydyn]
* ebfda13 - [aclorch] Fix TODO: use SAI object API to query capabilities (#2743) (2 days ago) [Stepan Blyshchak]
```
#### How I did it
#### How to verify it
#### Description for the changelog
yxieca pushed a commit that referenced this pull request Jun 16, 2023
…lly (#15508)

src/sonic-swss

* 444e3be - (HEAD -> 202205, origin/202205) [subinterface]: Fix admin state handling. (#2806) (10 hours ago) [Nazarii Hnydyn]
* 9690c5a - [portsorch]: Enhancing SWSS OA logs to capture host_tx_ready change events (#2822) (10 hours ago) [mihirpat1]
sonic-otn pushed a commit to sonic-otn/sonic-buildimage that referenced this pull request Sep 20, 2023
…lly (sonic-net#15468)

#### Why I did it
src/sonic-swss
```
* 87e0b08 - (HEAD -> master, origin/master, origin/HEAD) [portsorch]: Enhancing SWSS OA logs to capture host_tx_ready change events (sonic-net#2822) (11 hours ago) [mihirpat1]
* c7e52a0 - [subinterface]: Fix admin state handling. (sonic-net#2806) (34 hours ago) [Nazarii Hnydyn]
* ebfda13 - [aclorch] Fix TODO: use SAI object API to query capabilities (sonic-net#2743) (2 days ago) [Stepan Blyshchak]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants